Fixed bug related to UTF-8 in certain requests#93
Open
P3GLEG wants to merge 5 commits intoDataSploit:masterfrom
Open
Fixed bug related to UTF-8 in certain requests#93P3GLEG wants to merge 5 commits intoDataSploit:masterfrom
P3GLEG wants to merge 5 commits intoDataSploit:masterfrom
Conversation
Fixed bug in domain_censys.py not printing the given ip as well
Merging from master
Author
|
I added Dockerimages for general use and development as well. Love your tool! |
0xbadshah
suggested changes
Mar 12, 2017
0xbadshah
left a comment
0xbadshah
left a comment
There was a problem hiding this comment.
There is already a docker image (https://hub.docker.com/r/appsecco/datasploit/) available for Datasploit maintained by @madhuakula. The only difference is that his Dockerfile uses Ubuntu 16.04 docker image whereas you have used Debian Jessie
0xbadshah
reviewed
Mar 12, 2017
| && apt-get remove -y libxml2-dev python-dev libxmlsec1-dev curl wget build-essential \ | ||
| && rm -rf /var/lib/apt/lists/* | ||
|
|
||
| USER k4ch0w |
Author
There was a problem hiding this comment.
The user was left over from deving. My bad, I didn't see the dockerimage was already made. :( I will remove my image then and add his link to the documentation for new users of the project.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixed bug in domain_censys.py not printing the given ip as well